Handle rerun-if-changed directives in dependencies
authorAlex Crichton <alex@alexcrichton.com>
Wed, 13 Jan 2016 01:37:57 +0000 (17:37 -0800)
committerAlex Crichton <alex@alexcrichton.com>
Wed, 13 Jan 2016 01:44:18 +0000 (17:44 -0800)
commit7dcedd85a4c7d8bef6287d52fec204abb5573634
tree84f8a59961ba53a84194ef4bee4cf126102793c3
parent9b0ed1d80d48125ff4b0ef362309e7a1ed125ba8
Handle rerun-if-changed directives in dependencies

There was a failure mode of the handling of the rerun-if-changed directive where
it would rerun the build script twice before hitting a steady state of actually
processing the directives. The order of events that led to this were:

1. A project was built from a clean directory. Cargo recorded a fingerprint
   indicating this (for the build script), but the fingerprint indicated that
   the build script was a normal build script (no manually specified inputs)
   because Cargo had no prior knowledge.
2. A project was then rebuilt from the same directory. Cargo's new fingerprint
   for the build script now indicates that there is a custom list of
   dependencies, but the previous fingerprint indicates there wasn't, so the
   mismatch causes another rebuild.
3. All future rebuilds will agree that there are custom lists both before and
   after, so the directives are processed as one would expect.

This commit does a bit of refactoring in the fingerprint module to fix this
situation. The recorded fingerprint in step (1) is now recorded as a "custom
dependencies are specified" fingerprint if, after the build script is run,
custom dependencies were specified.

Closes #2267
src/cargo/ops/cargo_rustc/custom_build.rs
src/cargo/ops/cargo_rustc/fingerprint.rs
tests/test_cargo_freshness.rs
tests/test_cargo_rustc.rs